Skip to content

[3.7] bpo-31731: Fix test_io.check_interrupted_write() (GH-11225)#11226

Merged
miss-islington merged 1 commit into
python:3.7from
miss-islington:backport-05c9d31-3.7
Dec 18, 2018
Merged

[3.7] bpo-31731: Fix test_io.check_interrupted_write() (GH-11225)#11226
miss-islington merged 1 commit into
python:3.7from
miss-islington:backport-05c9d31-3.7

Conversation

@miss-islington

@miss-islington miss-islington commented Dec 18, 2018

Copy link
Copy Markdown
Contributor

Fix a race condition in check_interrupted_write() of test_io:
create directly the thread with SIGALRM signal blocked,
rather than blocking the signal later from the thread. Previously, it
was possible that the thread gets the signal before the signal is
blocked.
(cherry picked from commit 05c9d31)

Co-authored-by: Victor Stinner vstinner@redhat.com

https://bugs.python.org/issue31731

Fix a race condition in check_interrupted_write() of test_io:
create directly the thread with SIGALRM signal blocked,
rather than blocking the signal later from the thread. Previously, it
was possible that the thread gets the signal before the signal is
blocked.
(cherry picked from commit 05c9d31)

Co-authored-by: Victor Stinner <vstinner@redhat.com>

@vstinner vstinner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, good bot.

@miss-islington

Copy link
Copy Markdown
Contributor Author

@vstinner: Status check is done, and it's a success ✅ .

@miss-islington miss-islington merged commit 729fc5d into python:3.7 Dec 18, 2018
@miss-islington miss-islington deleted the backport-05c9d31-3.7 branch December 18, 2018 23:10
@miss-islington

Copy link
Copy Markdown
Contributor Author

@vstinner: Status check is done, and it's a success ✅ .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tests Tests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants